feat(google-slides): complete API surface for branded slide generation#4678
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
PR SummaryMedium Risk Overview Adds corresponding tool implementations (e.g. Reviewed by Cursor Bugbot for commit 2b11ce2. Configure here. |
Greptile SummaryThis PR adds 38 new Google Slides tools covering the full Slides REST API surface (text/paragraph styling, image replacement, shape/page/slide properties, transforms, z-order, grouping, lines, the complete table API, Sheets chart embedding, video, raw batchUpdate, Drive copy, and multi-format export), wiring all 38 operations into the existing block alongside a new shared
Confidence Score: 4/5The 38 new tool files and their block wiring are correct; the single flaw is the wand assistant on the raw batch_update field. Across ~8900 lines of new code the param remapping, color conversion, EMU math, and field-mask logic are all correct. The one defect is the requestsJson wand config using generationType json-object, which adds a system instruction telling the AI to produce object output — exactly what the batchUpdateTool rejects. Any user who invokes the wand to fill in the batch request array will receive an unusable value and a runtime error. apps/sim/blocks/blocks/google_slides.ts — specifically the requestsJson subBlock wandConfig around the batch_update operation. Important Files Changed
Reviews (2): Last reviewed commit: "fix(google-slides): declare z-order oper..." | Re-trigger Greptile |
…t base64 export, remove dead utility
|
@greptile |
|
@cursor review |
There was a problem hiding this comment.
✅ Bugbot reviewed your changes and found no new issues!
Comment @cursor review or bugbot run to trigger another review on this PR
Reviewed by Cursor Bugbot for commit 2b11ce2. Configure here.
Summary
tools/google_slides/utils.tswith helpers for color conversion, text-range/cell-location building, EMU dimensions, FieldMask construction, and standard URL/header builders.Type of Change
Testing
Tested manually. Validated wire shapes for every new tool against the official Slides v1 Discovery doc and Drive v3 reference via multiple parallel validation passes.
bun run type-check,bun run check:api-validation, andbun run lintall pass clean.Checklist